Skip to content

usermodel#78

Open
RoHanh96 wants to merge 1 commit intohellovietnam93:nguyen_van_hanh_20141418from
RoHanh96:modeluser
Open

usermodel#78
RoHanh96 wants to merge 1 commit intohellovietnam93:nguyen_van_hanh_20141418from
RoHanh96:modeluser

Conversation

@RoHanh96
Copy link

No description provided.

@RoHanh96 RoHanh96 force-pushed the modeluser branch 2 times, most recently from ab19ba3 to 8c5e2c9 Compare March 20, 2017 12:51
@@ -0,0 +1,3 @@
# Place all the behaviors and hooks related to the matching controller here.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file này k dùng thì xóa đi nhé

@@ -0,0 +1,3 @@
// Place all the styles related to the Users controller here.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

file cùng xóa đi

@@ -0,0 +1,2 @@
module UsersHelper
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fiel này cũng xóa đi

validate :check_phone
validate :check_date
has_secure_password
attr_accessor :test
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cái này có dùng ở đâu không

private
def check_phone
if self.phone
if (self.phone.length==10 && self.phone[0]=="0")
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

xóa () + viết cách nhau 1 dâu cách nhé

config/routes.rb Outdated
@@ -1,4 +1,6 @@
Rails.application.routes.draw do
get 'users/new'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sử dung nháy kép nhé

@@ -1,10 +1,10 @@
// This is a manifest file that'll be compiled into application.js, which will include all the files
// This is a manifest file that"ll be compiled into application.js, which will include all the files
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout file này ra mấy chỗ này là comment không cần sửa làm j cả

*
* Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
* or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
* or any plugin"s vendor/assets/stylesheets directory can be referenced here using a relative path.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tương tự checkout file này ra khỏi pull

@@ -1,5 +1,5 @@
module ApplicationHelper
def full_title page_title=''
def full_title page_title=""
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check lại ident trong file này đi nhé

end
end
end
end No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thiếu end file, thêm 1 dòng trống ở tân jcungf nhé


<%= stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track': 'reload' %>
<%= javascript_include_tag 'application', 'data-turbolinks-track': 'reload' %>
<%= stylesheet_link_tag "application", media: "all", "data-turbolinks-track": "reload" %>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sử dụng 1 dấu cách để ngăn

@@ -1,5 +1,5 @@
# Load the Rails application.
require_relative 'application'
require_relative "application"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout ra khỏi pull


# Enable/disable caching. By default caching is disabled.
if Rails.root.join('tmp/caching-dev.txt').exist?
if Rails.root.join("tmp/caching-dev.txt").exist?
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkou ra khỏi pull

# config.action_dispatch.x_sendfile_header = 'X-Sendfile' # for Apache
# config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for NGINX
# config.action_dispatch.x_sendfile_header = "X-Sendfile" # for Apache
# config.action_dispatch.x_sendfile_header = "X-Accel-Redirect" # for NGINX
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout ra khỏi pull

# test suite. You never need to work with it otherwise. Remember that
# your test database is "scratch space" for the test suite and is wiped
# and recreated between test runs. Don't rely on the data there!
# and recreated between test runs. Don"t rely on the data there!
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout ra khỏi pull

# If you change this key, all old signed cookies will become invalid!

# Make sure the secret is at least 30 characters and all random,
# no regular words or you'll be exposed to dictionary attacks.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

xóa file này bằng lệnh git rm -r --cache và cho vòa gitigonore để không pushs lên

LICENSE Outdated

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
of this software and associated documentation files (the 'Software'), to deal
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout ra khỏi pull đi em ơi

Rakefile Outdated
# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.

require_relative 'config/application'
require_relative "config/application"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout ra khỏi pull

module ApplicationHelper
def full_title page_title=''
base_title="Ruby on Rails Tutorial Sample App"
def full_title page_title = ""
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ident check lại đi em ơi

</div>
</header> No newline at end of file
</header>

No newline at end of file
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

để dòng blank nhé, k phải là thêm dòng có dấu cách


<%= stylesheet_link_tag 'application', media: 'all', 'data-turbolinks-track': 'reload' %>
<%= javascript_include_tag 'application', 'data-turbolinks-track': 'reload' %>
<%= stylesheet_link_tag "application", media: "all", "data-turbolinks-track": "reload" %>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 dấu cashc để ngăn em ơi, sau cái tag kìa

db/seeds.rb Outdated
# movies = Movie.create([{ name: 'Star Wars' }, { name: 'Lord of the Rings' }])
# Character.create(name: 'Luke', movie: movies.first)
# movies = Movie.create([{ name: "Star Wars" }, { name: "Lord of the Rings" }])
# Character.create(name: "Luke", movie: movies.first)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout

public/404.html Outdated
<html>
<head>
<title>The page you were looking for doesn't exist (404)</title>
<title>The page you were looking for doesn"t exist (404)</title>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout

public/422.html Outdated
<div>
<h1>The change you wanted was rejected.</h1>
<p>Maybe you tried to change something you didn't have access to.</p>
<p>Maybe you tried to change something you didn"t have access to.</p>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout

public/500.html Outdated
<html>
<head>
<title>We're sorry, but something went wrong (500)</title>
<title>We"re sorry, but something went wrong (500)</title>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout

@@ -1,4 +1,4 @@
require 'test_helper'
require "test_helper"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout

@RoHanh96 RoHanh96 force-pushed the modeluser branch 8 times, most recently from 45410bd to 9fd634c Compare March 21, 2017 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments